Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

do not merge: testing CI with all DB / logs changes #876

Closed
wants to merge 70 commits into from
Closed

do not merge: testing CI with all DB / logs changes #876

wants to merge 70 commits into from

Conversation

darrenvechain
Copy link
Member

No description provided.

paologalligit and others added 30 commits September 24, 2024 09:57
@paologalligit
feat: add new txIndex column to event meta response
a406f7f
@paologalligit
test: add convert event test
e22616b
@paologalligit
feat: make txLog and txIndex as optional return params
856701b
@paologalligit
chore: update swagger with new event optional data
37920c1
@paologalligit
feat: save logIndex in sequence
c30b6a0
@paologalligit
feat: tweaked bits in sequence
99d5fff
@paologalligit
refactor: rename optional log meta field
5663cb4
@paologalligit
refactor: comments, yaml and txIndex counts
10a4bc0
@darrenvechain @paologalligit
Darren/logdb remove leading zeros (#865)
ca8b38b
@paologalligit
feat: add new txIndex column to event meta response
f9173e4
@paologalligit
test: add convert event test
f3bd272
@paologalligit
feat: make txLog and txIndex as optional return params
c5b3c63
@paologalligit
chore: update swagger with new event optional data
76a38d9
@paologalligit
feat: save logIndex in sequence
ad6204e
@paologalligit
feat: tweaked bits in sequence
1415b40
@paologalligit
refactor: rename optional log meta field
94f4070
@paologalligit
refactor: comments, yaml and txIndex counts
464f530
@paologalligit
rebase to master
9114c0b
@paologalligit
fix: remove stale struct
a80a62a
@paologalligit
add txIndex to returned logdb query
9217741
@paologalligit
reset to 0 eventCount and transferCount each receipt and write blockI…
297ffd5 
…d only once
@paologalligit
fix lint
9ff47cb
@paologalligit
rephrase logIndex description in yaml file
6f4c9ef
@paologalligit
refactor: use filter.Option instead of eventFilter.Option
f81bc85
@libotony
move includeIndexes to api
a2a74f8
@darrenvechain @libotony
Darren/logdb remove leading zeros (#865)
d3e7903
@qianbin @libotony
cmd/thor: update instance dir to v4
a7cf6fd
@qianbin @libotony
trie: implement varint-prefix coder
b81910e
@qianbin @libotony
deps: add github.com/qianbin/drlp
19c7810
@qianbin @libotony
trie: implement appendHexToCompact & compactLen
7edff9c
qianbin and others added 25 commits November 8, 2024 18:10
@qianbin @libotony
api: changes due to underlying pkg update
d3040c1
@qianbin @libotony
cmd/thor/pruner: rename pkg optimizer to pruner
aaa4b8f
@qianbin @libotony
cmd/thor: changes due to underlying pkg update
aa2f9a4
@qianbin @libotony
muxdb: abandon leaf filter
73213b2
@qianbin @libotony
cmd/thor/pruner: use smaller period when nearly synced
63d8853
@qianbin @libotony
muxdb: improve trie node path encoding
b66b4ed
@qianbin @libotony
trie: treat short nodes as standalone nodes when skipping hash
982e02e
@qianbin @libotony
cmd/thor: fix disablePrunerFlag not work
235c964
@qianbin @libotony
trie: improve refNode encoding/decoding
79ece8f
@qianbin @libotony
muxdb: improve history node key encoding
b6cb3f6
@qianbin @libotony
cmd/thor: adjust pruner parameters
bc552d6
@qianbin @libotony
build: fix test cases
f550058
@qianbin @libotony
lint: fix lint error
e1d7f4a
@qianbin @libotony
muxdb: fix ver encoding in node blob cache
6f40214
@qianbin @libotony
muxdb: add test cases for cache
55e7f76
@qianbin @libotony
runtime: fix test compile error
16eaa32
@libotony
make build and test pass after rebase
230ecf7
@libotony
add back pruner tests
c649d54
@libotony
add tests for node encoding
29f904a
@libotony
minor typo
b7a42b3
@libotony
update named store space prefix
ae23b6d
@libotony
add more tests
7b883f1
@libotony
fix block summary in repo
67da142
@darrenvechain
merge branch 'feat/tx-log-seq' into feat/maindb-v4-candidate-pre-logdb
ca14f79
@darrenvechain
Merge branch 'feat/tx-log-seq' into feat/maindb-v4-candidate-pre-logdb
08def69
@darrenvechain darrenvechain requested a review from a team as a code owner November 8, 2024 12:56
@darrenvechain darrenvechain marked this pull request as draft November 8, 2024 12:56
@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 71.36007% with 657 lines in your changes missing coverage. Please review.

Project coverage is 61.30%. Comparing base (fd5e569) to head (08def69).
Report is 14 commits behind head on master.

Files with missing lines Patch % Lines
thorclient/thorclient.go 34.09% 86 Missing and 1 partial ⚠️
thorclient/httpclient/client.go 62.50% 47 Missing and 25 partials ⚠️
cmd/thor/pruner/pruner.go 62.33% 46 Missing and 12 partials ⚠️
muxdb/trie.go 65.11% 32 Missing and 13 partials ⚠️
trie/node.go 82.87% 34 Missing and 10 partials ⚠️
muxdb/cache.go 68.21% 39 Missing and 2 partials ⚠️
thorclient/wsclient/client.go 74.45% 32 Missing and 3 partials ⚠️
test/testchain/chain.go 71.30% 22 Missing and 11 partials ⚠️
state/state.go 66.66% 15 Missing and 5 partials ⚠️
tx/signer.go 48.64% 16 Missing and 3 partials ⚠️
... and 31 more
Additional details and impacted files 
@@            Coverage Diff             @@
##           master     #876      +/-   ##
==========================================
- Coverage   62.51%   61.30%   -1.22%     
==========================================
  Files         208      212       +4     
  Lines       18872    22225    +3353     
==========================================
+ Hits        11798    13625    +1827     
- Misses       5975     7506    +1531     
+ Partials     1099     1094       -5

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@darrenvechain darrenvechain deleted the feat/maindb-v4-candidate-pre-logdb branch November 26, 2024 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@darrenvechain @codecov-commenter @paologalligit @libotony @qianbin